Skip to content

Add explicit config flags for Tower user auto-generation#330

Merged
gwright99 merged 5 commits intomasterfrom
fix/issue-312-user-autogen-config
May 7, 2026
Merged

Add explicit config flags for Tower user auto-generation#330
gwright99 merged 5 commits intomasterfrom
fix/issue-312-user-autogen-config

Conversation

@gavinelder
Copy link
Copy Markdown
Contributor

Fixes #312

Summary

Adds two boolean variables to make Tower user auto-generation behavior explicit and configurable rather than hardcoded:

  • variables.tf — declares flag_tower_enable_participant_auto_create_user and flag_tower_enable_member_auto_create_user (both bool).
  • templates/TEMPLATE_terraform.tfvars — defaults both flags to false with an explanatory comment block clarifying the difference between Workspace collaborators (tower.participant.auto-create-user) and Organization members (tower.member.auto-create-user).
  • 009_define_file_templates.tf — passes both new vars into the tower_yml templatefile() call.
  • assets/src/tower_config/tower.yml.tpl — replaces the hardcoded participant.auto-create-user: false with the new variable and adds a new member: block driven by the second variable.

Automated triage by Claude Code.

gavinelder and others added 2 commits May 6, 2026 14:50
Introduces two boolean variables to make the Tower user auto-creation
behavior explicit and configurable rather than hardcoded:

- flag_tower_enable_participant_auto_create_user: governs
  tower.participant.auto-create-user (Workspace collaborators)
- flag_tower_enable_member_auto_create_user: governs
  tower.member.auto-create-user (Organization members)

Both default to false in TEMPLATE_terraform.tfvars. The previously
hardcoded participant flag in tower.yml.tpl now references the variable,
and a new member block is added.

Fixes #312

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
gwright99 added 3 commits May 7, 2026 14:09
- Added layer in base-overrides.auto.tfvars generation to set TEMPLATE_terraform.tfvars values from 'false' to 'true' (to align with existing test override pattern).
- Added overrides in each test of type "all-disabled".
- Allowed agent to ignore linter errors due to massive amounts of conflict between existing code and my strict local rules. Will fix in separate initiative.
@gwright99 gwright99 marked this pull request as ready for review May 7, 2026 19:24
@gwright99 gwright99 requested a review from schaluva May 7, 2026 19:24
@gwright99
Copy link
Copy Markdown
Member

@schaluva - another one (please).

@gwright99 gwright99 merged commit c20f9c4 into master May 7, 2026
4 checks passed
@gwright99 gwright99 deleted the fix/issue-312-user-autogen-config branch May 7, 2026 19:34
@gwright99 gwright99 linked an issue May 8, 2026 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ AI ] - Add guidance re: mandatory test framework updates [ Enhancement ] Add explicit config re: User auto-generation

3 participants